Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make minimal game duration to treat game as ranked configurable, lower default 180 -> 90 #271

Merged
merged 1 commit into from
Apr 16, 2024
Merged

Make minimal game duration to treat game as ranked configurable, lower default 180 -> 90 #271

merged 1 commit into from
Apr 16, 2024

Conversation

StanczakDominik
Copy link
Collaborator

@StanczakDominik StanczakDominik commented Apr 16, 2024

Fixes #214.

Relevant discussion in
https://discord.com/channels/549281623154229250/855772061095559179/1210540660604538951

Basically, some 1v1 games do legitimately conclude before 3:00, notably 2:33 is a reported today timing to shut down a comm drop. Also tick/rover rushes. This keeps coming up on the discord. 90s should be pretty optimal. And if it isn't, well, it's now configurable so we can just change the duration :)

Tested to not blow up the site, not tested in-game yet.

@StanczakDominik StanczakDominik marked this pull request as ready for review April 16, 2024 06:09
@StanczakDominik
Copy link
Collaborator Author

StanczakDominik commented Apr 16, 2024

Tested to work. I played two matches on the integration server, resigned at 60s in one of them, resigned at 120s in the other. Both show up in https://server5.beyondallreason.info/teiserver/admin/matches/user/44674 (which does not take ratings into account), only the latter shows up in https://server5.beyondallreason.info/teiserver/admin/users/ratings/44674 (which does).

Greenlighted by Borg

@Beherith Beherith merged commit b4178c0 into beyond-all-reason:master Apr 16, 2024
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 minute timer for ranked to count is way too long for 1v1 format.
2 participants